Skip to content

SQL: Add alias DATETRUNC to DATE_TRUNC function#47173

Merged
matriv merged 3 commits intoelastic:masterfrom
matriv:add-alias-datetrunc
Sep 27, 2019
Merged

SQL: Add alias DATETRUNC to DATE_TRUNC function#47173
matriv merged 3 commits intoelastic:masterfrom
matriv:add-alias-datetrunc

Conversation

@matriv
Copy link
Copy Markdown
Contributor

@matriv matriv commented Sep 26, 2019

To be on the safe side in terms of use cases also add the alias
DATETRUNC to the DATE_TRUNC function.

Follows: #46473

To be on the safe side in terms of use cases also add the alias
DATETRUNC to the DATE_TRUNC function.

Follows: elastic#46473
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-search

@matriv
Copy link
Copy Markdown
Contributor Author

matriv commented Sep 26, 2019

Not sure if it should be marked as enhancement (for release notes) or non-issue.
@costin ?

@matriv
Copy link
Copy Markdown
Contributor Author

matriv commented Sep 27, 2019

@elasticmachine run elasticsearch-ci/bwc

@costin
Copy link
Copy Markdown
Member

costin commented Sep 27, 2019

You could mark it as a feature since it's a new function.

Copy link
Copy Markdown
Member

@costin costin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@matriv matriv merged commit 9ac223c into elastic:master Sep 27, 2019
@matriv matriv deleted the add-alias-datetrunc branch September 27, 2019 12:38
matriv added a commit that referenced this pull request Sep 27, 2019
To be on the safe side in terms of use cases also add the alias
DATETRUNC to the DATE_TRUNC function.

Follows: #46473

(cherry picked from commit 9ac223c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants